Skip to content

Add copilot-instructions.md for GitHub copilot#16

Merged
genedna merged 3 commits into
libra-tools:mainfrom
genedna:main
Oct 25, 2025
Merged

Add copilot-instructions.md for GitHub copilot#16
genedna merged 3 commits into
libra-tools:mainfrom
genedna:main

Conversation

@genedna

@genedna genedna commented Oct 25, 2025

Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: Eli Ma <eli@patch.sh>
Copilot AI review requested due to automatic review settings October 25, 2025 11:11

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds GitHub Copilot custom instructions to guide AI-assisted development in the git-internal repository. The instructions establish standards for a Rust-based Git internals rewrite targeting monorepo-scale performance.

Key Changes:

  • Defines repository purpose: advanced Git object model rewrite for large-scale repositories
  • Establishes Rust coding standards, testing requirements, and performance considerations
  • Documents architectural components and Git compatibility requirements (SHA-1/SHA-256)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/copilot-instructions.md Outdated
Comment thread .github/copilot-instructions.md Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings October 25, 2025 11:14
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@genedna
genedna merged commit e605918 into libra-tools:main Oct 25, 2025

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment on lines +54 to +60
## Provide clear invariants

- Content-addressed: objects are identified by their content hash (SHA-1 or SHA-256).
- Idempotent: same input always produces same output.
- Backward/forward compatibility: old clients can fetch new packs, new clients can fetch old packs.

## Testing

Copilot AI Oct 25, 2025

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Duplicate section header: '## Provide clear invariants' appears at line 54 and its content is repeated within the '## Testing' section at lines 56-59. This creates confusion about the document structure. Consider either removing this standalone section and keeping the invariants content within the Testing section, or reorganizing to make the invariants a separate top-level section before Testing.

Suggested change
## Provide clear invariants
- Content-addressed: objects are identified by their content hash (SHA-1 or SHA-256).
- Idempotent: same input always produces same output.
- Backward/forward compatibility: old clients can fetch new packs, new clients can fetch old packs.
## Testing

Copilot uses AI. Check for mistakes.
genedna added a commit to genedna/git-internal that referenced this pull request Mar 15, 2026
* Add copilot-instructions.md for GitHub copilot

Signed-off-by: Eli Ma <eli@patch.sh>

* Update .github/copilot-instructions.md

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update .github/copilot-instructions.md

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

---------

Signed-off-by: Eli Ma <eli@patch.sh>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants